Skip to content

Persona - Separated SCSS for Persona and PersonaCoin into Two Files#4200

Merged
mtennoe merged 3 commits intomicrosoft:masterfrom
oengusmacinog-zz:persona-splitscss
Mar 7, 2018
Merged

Persona - Separated SCSS for Persona and PersonaCoin into Two Files#4200
mtennoe merged 3 commits intomicrosoft:masterfrom
oengusmacinog-zz:persona-splitscss

Conversation

@oengusmacinog-zz
Copy link
Copy Markdown
Collaborator

@oengusmacinog-zz oengusmacinog-zz commented Mar 7, 2018

Pull request checklist

  • Include a change request file using $ npm run change

Description of changes

Seperated the SCSS for Persona and PersonaCoin components into two SCSS files. Because the full 16KB Persona.scss file was imported in Persona and PersonaCoin is was causing some package bloat.

@lynamemi lynamemi requested a review from mtennoe March 7, 2018 00:11
Copy link
Copy Markdown
Contributor

@unindented unindented left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should make a big impact on our bundle size. Thanks!

@mtennoe
Copy link
Copy Markdown
Member

mtennoe commented Mar 7, 2018

Thank you so much for doing this!

@mtennoe mtennoe merged commit dfe67ea into microsoft:master Mar 7, 2018
@oengusmacinog-zz oengusmacinog-zz deleted the persona-splitscss branch March 7, 2018 17:27
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants